Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: update ibc-go to v4, adapt packages and API #8409

Merged
merged 7 commits into from
Nov 15, 2023
Merged

Conversation

JimLarson
Copy link
Contributor

@JimLarson JimLarson commented Sep 28, 2023

closes: #8223

Description

Upgrade to ibc-go v4.4.2. Entails upgrades of cosmos-sdk to v0.45.16 and tendermint to cometbft v0.34.27.

Security Considerations

Closes any security issues that these upgrades address.

Scaling Considerations

N/A

Documentation Considerations

N/A

Testing Considerations

See test plan document.

Upgrade Considerations

Yes.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! So much easier.

@mhofman
Copy link
Member

mhofman commented Nov 5, 2023

Can we merge agoric-labs/cometbft#5 first and only update the cometbft dependency once.

@@ -142,14 +155,11 @@ replace google.golang.org/grpc => google.golang.org/grpc v1.33.2

// At least until post-v0.34.14 is released with
// https://github.com/tendermint/tendermint/issue/6899 resolved.
replace github.com/tendermint/tendermint => github.com/agoric-labs/tendermint v0.34.23-alpha.agoric.3
replace github.com/tendermint/tendermint => github.com/agoric-labs/cometbft v0.34.27-alpha.agoric.2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New version has been tagged with the BlockParams.MaxBytes change

Suggested change
replace github.com/tendermint/tendermint => github.com/agoric-labs/cometbft v0.34.27-alpha.agoric.2
replace github.com/tendermint/tendermint => github.com/agoric-labs/cometbft v0.34.27-alpha.agoric.3

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@JimLarson JimLarson self-assigned this Nov 8, 2023
@JimLarson JimLarson added agoric-cosmos enhancement New feature or request labels Nov 8, 2023
@JimLarson JimLarson marked this pull request as ready for review November 10, 2023 02:26
@JimLarson
Copy link
Contributor Author

PTAL @michaelfig @mhofman

The backport referenced above has the agd process hang around after halting.
Wrote a wrapper utility which scans stderr for the halt application message,
interrupts the process, then propagates its error signal. Conventional shell
scripting is not up to the challenge of expressing this behavior.
…#305"

This reverts commit efcc3b0.

We decided to roll back the problematic change (agoric-labs/cosmos-sdk#305).
@JimLarson JimLarson added the automerge:rebase Automatically rebase updates, then merge label Nov 15, 2023
@mergify mergify bot merged commit 28931a7 into master Nov 15, 2023
77 checks passed
@mergify mergify bot deleted the 8223-ibc-4 branch November 15, 2023 09:50
mhofman pushed a commit that referenced this pull request Dec 6, 2023
feat: update ibc-go to v4, adapt packages and API
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agoric-cosmos automerge:rebase Automatically rebase updates, then merge enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to ibc-go v4
3 participants